Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AE-432] Add Nicla Vision to build script #11

Merged
merged 4 commits into from
Sep 2, 2024

Conversation

aliphys
Copy link
Contributor

@aliphys aliphys commented Aug 29, 2024

This PR adds the Nicla Vision to the example build script:

  • 🔍 The following three examples, common to all boards, are covered by the build script for Nicla Vision
    • examples/Battery
    • examples/Charger
    • examples/Standby_WakeFromPin
  • 📚 Currently, the Arduino_LowPowerNiclaVision library is not included in any of the examples nor source file. It is expected that this would change, given the compiler directives for the Nicla Vision e.g.
    void Board::setCameraPowerEnabled(bool on) {
    #if defined(ARDUINO_NICLA_VISION)
    if(on){
    PMIC.getControl()->turnLDO1On(Ldo1Mode::Normal);
    PMIC.getControl()->turnLDO2On(Ldo2Mode::Normal);
    PMIC.getControl()->turnLDO3On(Ldo3Mode::Normal);
    } else {
    PMIC.getControl()->turnLDO1Off(Ldo1Mode::Normal);
    PMIC.getControl()->turnLDO2Off(Ldo2Mode::Normal);
    PMIC.getControl()->turnLDO3Off(Ldo3Mode::Normal);
    }
    #endif
    }
  • 🔑 A PAT is used to access the Low Power library for the Nicla Vision (see here).

@aliphys aliphys added the topic: infrastructure Related to project infrastructure label Aug 29, 2024
@aliphys aliphys requested a review from sebromero August 29, 2024 07:40
@aliphys
Copy link
Contributor Author

aliphys commented Aug 29, 2024

Removed reference to library in afeb0f3 and example that depends on low power features in 41b4adb cc @sebromero

Copy link
Contributor

@sebromero sebromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sebromero sebromero merged commit 652f2b4 into arduino-libraries:main Sep 2, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants